Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reflect more complex local parameters #894

Merged
merged 3 commits into from
Feb 16, 2024

Conversation

Sustrak
Copy link
Contributor

@Sustrak Sustrak commented Feb 14, 2024

With this commit slang-reflect is able to reflect more complex local parameters. Also, it will throw an error if it doesn't know how to reflect the parameter instead of crashing.

Fixes #825

Sustrak and others added 2 commits February 14, 2024 20:49
With this commit slang-reflect is able to reflect more complex
local parameters. Also, it will throw an error if it doesn't know
how to reflect the parameter instead of crashing.
Copy link

codecov bot commented Feb 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2423734) 93.76% compared to head (59f0207) 93.76%.
Report is 12 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           master     #894    +/-   ##
========================================
  Coverage   93.76%   93.76%            
========================================
  Files         189      189            
  Lines       46866    47037   +171     
========================================
+ Hits        43942    44106   +164     
- Misses       2924     2931     +7     

see 23 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2423734...59f0207. Read the comment docs.

@MikePopoloski MikePopoloski merged commit f904c15 into MikePopoloski:master Feb 16, 2024
15 checks passed
@Sustrak Sustrak deleted the hotfix/slang-reflect-825 branch May 27, 2024 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong index variant on slang-reflect
2 participants